[PATCH] D113707: [clang] Make -masm=intel affect inline asm style

Nico Weber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Nov 17 14:06:28 PST 2021


thakis added a comment.

This contained a bug in that I didn't change CompilerInvocation::GenerateCodeGenArgs() to serialize the new flag correctly, so in build configs that do roundtripping the test failed.

I fixed the marshalling in https://reviews.llvm.org/rG36873fb768dbedeb3e9167117d3bb63b3720d214 (and https://reviews.llvm.org/rG3623163ae83799933bbeabe2cb3060537250c4b3), and I made https://reviews.llvm.org/D114120 to make sure more build configs do roundtripping (with that, I would've seen this locally instead of only on the bots).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D113707/new/

https://reviews.llvm.org/D113707



More information about the cfe-commits mailing list