myatsina added a comment.

In https://reviews.llvm.org/D15075#631207, @vitalybuka wrote:

> These patches break asan tests: 
> http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/528/steps/check-asan%20in%20gcc%20build/logs/stdio


Vitaly ,

This patch is a gcc compatibility issue and it changes clang to output an error 
in case of illegal inline assembly syntax related to clobber list.

Commit r290540 changed an asan test (asan_asm_test.cc) that used the illegal 
syntax and fixed it.
The commit removed from the extended inline assembly clobber list registers 
that also appeared in the  input list.
GCC fails as well on the original inline assembly that appeared in this test, 
so the fix is correct.
I don't understand why this change has effect the logic of the test - can you 
help?

Thanks,
Marina


Repository:
  rL LLVM

https://reviews.llvm.org/D15075



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to