[clang-tools-extra] issue-63565: community requested small QoL fix for more configurabili… (PR #108005)
via cfe-commits
cfe-commits at lists.llvm.org
Wed Sep 18 07:02:02 PDT 2024
================
@@ -6,6 +6,7 @@
//
//===----------------------------------------------------------------------===//
+#include "../Config.h"
----------------
MK-Alias wrote:
>This can be included without the ../
Yes, but it of-course shouldn't! Confirm the [standard](https://www.open-std.org/jtc1/sc22/wg14/www/docs/n1570.pdf#page=182) it should be either: `#include "../Config.h"` or `#include <Config.h>` (and also it should be `Config.hpp`)
But because this is used in the whole codebase, if changed it to `#include "Config.h"`
https://github.com/llvm/llvm-project/pull/108005
More information about the cfe-commits
mailing list