[PATCH] D97233: Support `#pragma clang section` directives on MachO targets

[Tim Northover via Phabricator via llvm-commits]

t.p.northover added inline comments.


================
Comment at: clang/lib/Sema/SemaAttr.cpp:304
 
+  if (llvm::Error E = Context.getTargetInfo().isValidSectionSpecifier(SecName))
+    Diag(PragmaLoc, diag::err_pragma_section_invalid_for_target)
----------------
Shouldn't this block return so the section doesn't get marked as valid later?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97233