[Bug bootstrap/63888] [5 Regression] bootstrap failed when configured with -with-build-config=bootstrap-asan --disable-werror

2015-02-04 Thread echristo at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63888 --- Comment #30 from Eric Christopher echristo at gmail dot com --- That might be reasonable. As far as I know ld64 doesn't do that yet, but I know it's been thought about.

[Bug bootstrap/63888] [5 Regression] bootstrap failed when configured with -with-build-config=bootstrap-asan --disable-werror

2015-02-03 Thread echristo at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63888 --- Comment #26 from Eric Christopher echristo at gmail dot com --- Sure. Not the general case for other targets though.

[Bug bootstrap/63888] [5 Regression] bootstrap failed when configured with -with-build-config=bootstrap-asan --disable-werror

2015-02-03 Thread echristo at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63888 --- Comment #28 from Eric Christopher echristo at gmail dot com --- Ah, but not every platform is ELF :) ld64 has the flexibility to do this with Mach-O. As I said, I don't have any better ideas at the moment, but warning that it is possible

[Bug bootstrap/63888] [5 Regression] bootstrap failed when configured with -with-build-config=bootstrap-asan --disable-werror

2015-02-03 Thread echristo at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63888 --- Comment #23 from Eric Christopher echristo at gmail dot com --- So, I think Jakub's solution is strictly better here as it allows intermixing of asan and non-asan code. It'll involve a bit of work in llvm's middle end to keep track of symbol

[Bug bootstrap/63888] [5 Regression] bootstrap failed when configured with -with-build-config=bootstrap-asan --disable-werror

2015-02-03 Thread echristo at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63888 --- Comment #24 from Eric Christopher echristo at gmail dot com --- For the record btw, I don't believe there's a reason why the linker couldn't split up the data section by knowing the size of the variable so it's worth being very careful here

[Bug c/60490] please define __LITTLE_ENDIAN__/__BIG_ENDIAN__ for every target where it makes sense

2014-03-10 Thread echristo at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60490 --- Comment #2 from Eric Christopher echristo at gmail dot com --- Why does it seem like a bad decision? Endianness can be separate from OS (or bare metal) so I don't see how outputting the macro as a per-cpu define is a bad thing.

[Bug c/60490] please define __LITTLE_ENDIAN__/__BIG_ENDIAN__ for every target where it makes sense

2014-03-10 Thread echristo at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60490 --- Comment #4 from Eric Christopher echristo at gmail dot com --- I disagree for bare metal that including endian is the right way, but agree that __BYTE_ORDER__ is the right way to do this in general. Thanks Jakub.

[Bug libitm/55693] [4.8 Regression] libitm.c++/eh-1.C execution test fails on darwin from r193271

2013-01-17 Thread echristo at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55693 Eric Christopher echristo at gmail dot com changed: What|Removed |Added CC